Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Corrected type on bufputc #124

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Corrected type on bufputc #124

wants to merge 1 commit into from

Conversation

andre-d
Copy link
Contributor

@andre-d andre-d commented Jul 25, 2012

Possible vulnerability implicit cast, my bad

@andre-d
Copy link
Contributor Author

andre-d commented Jul 25, 2012

Might be implicitly casted, good idea to correct anyway.

@andre-d
Copy link
Contributor Author

andre-d commented Jul 25, 2012

Confirmed, implicit cast to char, still likely should be uint8_t or char to begin with.

@mildsunrise
Copy link

@andre-d 👍 anyway, your changes can't cleanly merge with master because they've different base commits.
If you can add me to your repo, I'll fix this for you. Thanks!

@aidanhs aidanhs mentioned this pull request Oct 8, 2013
Closed
GerHobbelt pushed a commit to GerHobbelt/soldout that referenced this pull request Feb 11, 2021
Merge pull request vmg#124 from jmendeth/renderer-api
758951b 
Renderer API
dokutoku pushed a commit to dokutoku/sundown that referenced this pull request Mar 11, 2023
Merge vmg#124 of the original project
6fd5f76
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@andre-d @mildsunrise